Skip to content

Remove paths-ignore filters from required workflows - #349

Merged
afritzler merged 1 commit into
ironcore-dev:mainfrom
gonzolino:chore/path-filters
Jul 16, 2026
Merged

Remove paths-ignore filters from required workflows#349
afritzler merged 1 commit into
ironcore-dev:mainfrom
gonzolino:chore/path-filters

Conversation

@gonzolino

@gonzolino gonzolino commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Proposed Changes

  • Remove path filters from required workflows. The filters prevent PRs from
    merging if they do not match the filter.

Summary by CodeRabbit

  • Chores
    • CI workflows now run for pull requests that modify documentation or Markdown files.
    • Linting, testing, and Docker publishing checks consistently trigger across all pull requests.

@gonzolino
gonzolino requested a review from a team as a code owner July 16, 2026 11:40
@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3143ab4e-d5bd-41d7-888c-db740a178e11

📥 Commits

Reviewing files that changed from the base of the PR and between 6e3fcc5 and 87b7b7f.

📒 Files selected for processing (3)
  • .github/workflows/lint.yml
  • .github/workflows/publish-docker.yml
  • .github/workflows/test.yml
💤 Files with no reviewable changes (2)
  • .github/workflows/publish-docker.yml
  • .github/workflows/test.yml

📝 Walkthrough

Walkthrough

The lint, Docker publish, and test GitHub Actions workflows now run for pull requests that modify documentation or Markdown files by removing their paths-ignore filters.

Changes

Workflow trigger coverage

Layer / File(s) Summary
Remove pull request path exclusions
.github/workflows/lint.yml, .github/workflows/publish-docker.yml, .github/workflows/test.yml
The workflows no longer exclude docs/** and **/*.md changes from pull request triggers; existing event types remain configured.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

Suggested labels: chore, area/metal-automation, size/S

Suggested reviewers: atd9876

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: removing paths-ignore filters from required workflows.
Description check ✅ Passed The description follows the template's main section and explains the change, with only the Fixes # reference omitted.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: Daniel Gonzalez Nothnagel <daniel.gonzalez.nothnagel@sap.com>
@gonzolino
gonzolino force-pushed the chore/path-filters branch from 92deecd to 87b7b7f Compare July 16, 2026 12:15
@afritzler
afritzler merged commit ab490f4 into ironcore-dev:main Jul 16, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this to Done in Roadmap Jul 16, 2026
@gonzolino
gonzolino deleted the chore/path-filters branch July 16, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants